Skip to content

Lock the version of the spawned server #402

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 8, 2015
Merged

Lock the version of the spawned server #402

merged 1 commit into from
May 8, 2015

Conversation

TimMoore
Copy link
Contributor

This fixes a bug when the currently-executing version of Spring is not the most recent version available in the gem path. Previously, this would launch the most recent server version and cause an error when Bundler detects the mismatch.

Fixes #295.

@TimMoore
Copy link
Contributor Author

Any thoughts about writing a test for this? It's a little tricky to add a regression test, because the bug only manifests when you're running an older version of the gem than the most recent version installed.

@TimMoore
Copy link
Contributor Author

TimMoore commented May 1, 2015

Is there any interest in this change from the maintainers? I'm happy to rebase the branch and fix the merge conflicts if so.

@jonleighton
Copy link
Member

Hi @TimMoore, sorry for the slowness. This change looks good to me. Please do rebase and I'll get it released.

This fixes a bug when the currently-executing version of Spring is not
the most recent version available in the gem path. Previously, this
would launch the most recent server version and cause an error when
Bundler detects the mismatch.

Fixes #295.
@TimMoore
Copy link
Contributor Author

TimMoore commented May 3, 2015

Thanks, @jonleighton. Done!

jonleighton added a commit that referenced this pull request May 8, 2015
@jonleighton jonleighton merged commit b692823 into rails:master May 8, 2015
NuckChorris added a commit to hummingbird-me/kitsu-tools that referenced this pull request Jun 6, 2015
NuckChorris added a commit to hummingbird-me/kitsu-tools that referenced this pull request Jun 6, 2015
NuckChorris added a commit to hummingbird-me/kitsu-tools that referenced this pull request Jun 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Client-server version mismatch when multiple versions of spring installed for the current Ruby
2 participants